toolbar: Replace button-press-event handler with gesture

linkbutton: Replace button-press-event handler with gesture

separatortoolitem: Remove button-event/motion-event handlers

These only exist for the window dragging which does not exist anymore
currently. It will be reintroduced later in a form that does not require
these handlers.

snapshot: Add annotations to parameters/return values

widget: Use gtk_widget_measure to measure widget sizes

widget: Remove _gtk_widget_supports_clip

widget: Remove gtk_widget_invalidate_style_context

wayland: Sync attributes also when drawing with GL

Probably syncing attributes shouldn't happen when drawing in the first
place, but what do I know about Wayland. ¯\_(ツ)_/¯

wayland: Uncomment erroneously commented out code

I read the code as if (use_gl) instead of if (!use_gl) and commented it
out in bddfd7bb41cebd9b4e8ea418ce4d8e1f11822e81. That broke drawing on
Wayland without OpenGL completely.

Whoops.

Now it's back.

GDK/Win32: Fix build after GDKGL refactoring

There were some parts that need some updates after the refactoring in
GDKGL, so that the code will continue to build and run.

For gdkwindow-win32.c, comment out the parts where we check for use_gl
(which was removed), since we are going to move all drawing to OpenGL,
but don't remove/disable the whole portion as that transition is not
complete at this point.

There a is new GDKGL function that checks for the damaged area of the back
buffer, but since the notion of "damage" is for *NIX (GLX/EGL for
Wayland/mir), meaning that there is no such extension for Windows in this
regard, so we can't support this on Windows as-is, at least for now.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Update POTFILES.in

build: add gir depends for GdkX11, GdkWin32, and Gsk on Gdk

https://bugzilla.gnome.org/show_bug.cgi?id=775651

mir: fix type warning

mir: fix build failures

mir: implement display monitor vfuncs

gdkgl: Add gdk_gl_context_get_damage()

This is a way to query the damaged area of the backbuffer.

The GL renderer uses this to compute the extents of that damage region
(computed via buffer age) and use them to minimize the area to redraw.

This changes the semantics of GL rendering to "When calling
gdk_window_begin_frame() with a GL context, the area by
gdk_gl_context_get_damage() needs to be redrawn and every other pixel of
the backbuffer is guaranteed to be correct.
After gdk_window_end_frame() on a GL-drawn window, the whole backbuffer
must be correct.

We can always glXBufferSwap() now because of this.

inspector: Add support for displayin the render region

Now I just need a UI designer that makes this look nice...

glrenderer: Always redraw the whole screen

gdk: Make gdk_window_begin_draw_frame() take a draw context

... instead of a gl context.

This requires some refactoring in the way we mark the shared context as
drawing: We now call begin_frame/end_frame() on it and ignore the call
on the main context.
Unfortunately we need to do this check in all vfuncs, which sucks. But I
haven't found a better way.

gskrenderer: Add gsk_renderer_begin_draw_frame()

This way, we can hijack the begin/end draw process and do out own
processing before passing it on to GDK.

inspector: Pass the drawing context into the recorder

That way we can capture both the actual changes (clip region) and the
area that was redrawn (render region), which in OpenGL might not be
identical.

Nothing shows the render region yet though...

glrenderer: Always redraw the whole screen

gsk: Change GSK_USE_SOFTWARE env var to GSK_RENDERER

This way, we can use the variable as more than a boolean flag.

This will be useful for Vulkan or D3D renderers.

gdk: Turn GdkGLContext into a GdkDrawContext

gdk: Add GdkDrawContext

This will be the base class for GdkVulkanContext and GdkGLContext.

gdk: Redo GL drawing

Reenable GL drawing, but do it without Cairo.

Now, the context passed to gdk_window_begin_draw_frame() decides how
drawing is going to happen. If it is NULL, Cairo is used like before.

If a context is passed, Cairo may not be used for drawing and
gdk_drawing_context_get_cairo_context() is going to return NULL.
Instead, the GL renderer must draw to the GL backbuffer and
end_draw_frame() is then swapping that to the front.

The GskGLRenderer has lost the texture it used to render to and adapted
to render directly to the backbuffer instead.

The only thing missing is for GtkGLArea to gain back a performant way to
render. But it didn't have one since the introduction of GSK, this
patchset doesn't change anything about it.

The new rendering avoids two indirections (the GSK renderer's texture
and the GDK double buffering surface).
It improves icon count in the fishbowl demo by 30%.

glcontext: Make begin/end_draw() paired

This way, we can query the GL context's state via
gdk_gl_context_is_drawing().

Use this function to make GL contexts as attached and grant them access
to the front/backbuffer for rendering.

All of this is still unused because GL drawing is still disabled.

gdk: Large GL refactoring

No visible changes as GL rendering is disabled at the moment.

What was done:

1. Move window->invalidate_for_new_frame to glcontext->begin_frame
This moves the code to where it is used (the GLContext) and prepares it
for being called where it is used when actually beginning to draw the
frame.

2. Get rid of buffer-age usage
We want to let the application render directly to the backbuffer.
Because of that, we cannot make any assumptions about the contents the
application renders outside the clip area.
In particular GskGLRenderer renders random stuff there but not actual
contents.

3. Pass the actual GL context
Previously, we passed the shared context to end_frame, now we pass the
actual GL context that the application uses for rendering. This is so
that the vfuncs could prepare the actual contexts for rendering (they
don't currently).

4. Simplify the code
The previous code set up the final drawing method in begin_frame.
Instead, we now just ensure the clip area is something we can render
and decide on the actual method in end_frame.
This is both more robust (we can change the clip area in between if we
want to) and less code.

gsk: Allow 0 for gsk_gl_driver_begin_frame()

Special-case this situation to unset the current framebuffer and render
to the backbuffer again.

gdk: Don't recurse when processing updates

We don't need to send expose events for backwards compatibility anymore.

gdk: Remove all code that only existed because of use_gl

Now that we don't use GL anymore, this code is unnecessary.

gdk: Never draw with GL

This is a temporary switch-off of the GL dawing code that will make
things keep running. All GL related code (like the GSK renderer or
GtkGLArea will now fall back to software.

gskrenderer: Store the GL context

And use it to create the drawing context with it.

Note that this doesn't yet have any effect and is all infrastructure
preparation work.

API: Require passing a GLContext to begin_draw_frame()

This is in preparation for requiring explicit passing of GL contexts
when drawing.

window: Change behavior of gdk_window_begin_draw_frame()

(1) Require a native window
(2) Create the drawing context before calling begin_paint().

drawingcontext: Store the GL paint context

cssgadget: Compute relative to own allocation, not widget allocation

This would lead to clipped gadgets when the gadget allocation differed
from the widget allocation, like in GtkListBox.

wayland: destroy subsurfaces along with parents

Wayland subsurfaces can have other native window parents, but those need
to be destroyed along with the rest of the window hierarchy otherwise
an assert() is reached.

https://bugzilla.gnome.org/show_bug.cgi?id=774915

frame: Use gtk_widget_measure to measure child sizes

viewport: Use gtk_widget_measure to measure child sizes

and rewrite the adjustment handling in a orientation-agnostic way.

scrolledwindow: measure child widgets using gtk_widget_measure

revealer: Measure child widget size using gtk_widget_measure

headerbar: measure children using gtk_widget_measure

widget-factory: Use GtkButton:icon-name to construct icon buttons

toolpalette: call set_has_window

printunixdialog: Remove GtkDrawingArea::draw handler

the source file has been updated to use gtk_drawing_area_set_draw_func,
but the draw_cb handler in the ui file is still left.

stylecontext: Remove a bunch of unneeded headers

GtkProgressBar: trivial conditional optimisation

I'd hope the compiler would realise this for us, but let's be explicit.

entry: Refactor get_icon_pixbuf

This way it will only return a pixbuf if the icon helper has a pixbuf.

iconhelper: Ensure the surface used for textures is ARGB32

iconhelper: Remove allocate implementation

listbox: Fix row hovering

GtkListBox is not a windowed widget anymore so we can't use
gtk_widget_get_window. Just directly access priv->view_window instead to
get the right window.

spinbutton: Remove _get_panels

Remove gtk_widget_get_preferred_height_and_baseline_for_width

It's just a wrapper around gtk_widget_measure nowadays.

togglebutton: Remove unused defines

flowbox: Fix get_child_at_index crash with an invalid index

https://bugzilla.gnome.org/show_bug.cgi?id=775525

ComboBox: Fix the whitespace fix

ComboBox: Do not select item before menu realised

For a menu mode CB with wrap_width == 0 and an active item, that item is
selected in gtk_combo_box_menu_popup. Selection causes the MenuShell to
activate and hence take a grab. This was done before the menu was popped
up. A patch distributed in Debian sid - after being proposed on our BZ -
revealed that on the 1st popup of any such ComboBox, within grab_add,
the MenuShell's toplevel's GdkWindow is NULL. This causes a Gdk-CRITICAL
assertion fail on the 1st time opening any such CB, on Debian and if
that patch were merged to GTK+. By selecting after popup, we ensure the
MenuShell is realised before its grab_add and so avoid the critical.

https://bugzilla.gnome.org/show_bug.cgi?id=771242

ComboBox: Fix whitespace

 * Replace tabs for indentation with spaces
 * Remove whitespace at ends of lines

cssshadows: Fix extents computation

Someody busted this code a *lot*.

image: Don't translate twice

The animation can pass x/y through to the render_icon() function, so
don't gtk_snapshot_translate() it.

wayland: Don't warn if we loose the compositor connection

And instead, exit cleanly. This avoids filling the logs with
these warnings from every single application that has a
connection to the compositor.

x11: Don't warn if the display is closed

This causes a storm of warnings from all applications in the logs
whenever the display goes away, and is not useful.

gsk: ensure libgsk-4.la before running g-ir-scanner

https://bugzilla.gnome.org/show_bug.cgi?id=775410

gskrenderer: Add GError argument to gsk_renderer_realize()

This way, we don't spam criticals when GL is not available. Instead, we
print a useful debug message to stderr and continue with the Cairo renderer.

Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>

gsk: Add gsk_renderer_new_for_window()

and remove gsk_renderer_get_for_display().

This new function returns a realized renderer. Because of that, GSK can
catch failures to realize, destroy the renderer and try another one.

Or in short: I can finally use GTK on Weston with the nvidia binary
drivers again.

Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>

gsk: Change gsk_renderer_realize()

Instead of having a gsk_renderer_set_window() call, pass the window to
realize(). This way, the realization can fail with the wrong window.

Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>

gdk: Make GdkDrawingContext not per-backend

No backend is using it, and we can put the backend-specific drawing code
into GdkGLContext.

gdkwindow: Remove unused vfunc

GdkGLContext knows how to realize itself.

flowbox: Size gadget realtive to widget->window

This fixes a few clipping issues and syncs code with GtkListBox.

listbox: Move the priv->gadget allocation

It used to be relative to the window, now it is relative to the widget.

That is necessary so the snapshot fuction doesn't confuse coordinate
systems.

cairorenderer: Remove nonexisting function from header

wayland: Check for subsurface looking up the toplevel

gdk_window_get_toplevel() walks up the windows tree looking for the
corresponding toplevel window, but needs to account for subsurfaces as
well on Wayland.

https://bugzilla.gnome.org/show_bug.cgi?id=775319

demos: Remove undefined function call

The gtk_container_set_border_width() function has been removed from
the GTK+ 4.x API.

gtk-demo: Add a demo for tabs

One of the least-appreciated features in pango.
It deserves a demo.

Don't leak a pixbuf reference in dnd

https://bugzilla.gnome.org/show_bug.cgi?id=775316

Fix reference handling in GtkScaleButton

We were leaking the adjustment, since we confuse ourselves
with a property whose initial value comes out of a template.
Stop doing that.

https://bugzilla.gnome.org/show_bug.cgi?id=775212

mir: fix build failures

docs: fix a parameter name of GtkEntry::populate-popup

Trivial commit.

The documentation block refers to @widget, not @popup. @widget is a
better name since the type is GtkWidget.

docs: fix docs of functions to convert layout_index <-> text_index

Trivial commit.

The documentation was swapped. The documentation for the parameters and
the return values is good.

rendericon: Pass the scale factor when rendeirng textures

Fixes icon rendeirng on hidpi.

Adwaita: make rubberband selection work again for libgd apps

libgd views still use the old style class.

Make gtk-encode-symbolic-svg work for icons with dotted names

We were producing org.symbolic.png from org.gnome.Recipes-symbolic.svg,
which is not useful. Look for the last dot in the original name, to
produce the expected org.gnome.Recipes-symbolic.symbolic.png instead.

Build: Add wayland to GSKs dependencies

gskrenderer.c includes gdk/wayland/gdkwayland.h and as a consequence
we need to be able to locate wayland's headers in case they are not
in standard location.

https://bugzilla.gnome.org/show_bug.cgi?id=775038

gdk: Rewrite docs some more

Now gdk_window_process_updates() isn't mentioned either.

GDK takes care of redrawing, not you!

inspector: Remove leftover GDK_PRIVATE_CALL()s

gdk: Get rid of all mentions of process_all_updates()

wayland: Place subsurfaces relative to their parent

Now that subsurfaces can be created as child of another GdkWindow (and
not just the root window), they must be placed according to the location
of their parent, i.e. the abs_x/abs_y must be updated and taken int
account when placing and moving subsurfaces under Wayland.

https://bugzilla.gnome.org/show_bug.cgi?id=774917

gdk: Use stricter checks for begin_draw_frame()

We don't support drawing on child windows, so make sure nobody tries.

Drop the GDK_PRIVATE_CALL mechanism

We don't need to do this extra complication anymore, since
we're now living inside a single shared library.

inspector: Respect text-scaling-factor value initially

This was pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=774893

GtkLabelAccessible: Initialize link before setting parent

Since at-spi-atk commit 96621a5e95 fixed PropertyChange notifications
for AccessibleParent, setting the parent will result in a call to
ref_state_set() which assumes that the object is fully initialized.

https://bugzilla.gnome.org/show_bug.cgi?id=774939

inspector: ensure controller is a GtkGesture

While GtkEventController implementations today are all GtkGesture, it is
possible to create a GtkEventController manually. This is an extrac check
to ensure we only add gestures to the list.

https://bugzilla.gnome.org/show_bug.cgi?id=774760

Update Russian translation
(cherry picked from commit dd3cf38c53239cdd93f5ff887120ee453292fe5d)

menu: Don't leak check menu items

Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=774686

notebook: Don't leak arrow gadgets

This was pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=774743

text handle: Don't leak adjustments

This can happen if the weak pointer is triggered before the
adjustments are unset. Pointed out in

https://bugzilla.gnome.org/show_bug.cgi?id=774790

gdk: Only draw toplevels

Simplify gdk_window_paint_on_clock () to only draw the toplevel. We do
no longer draw child windows and offscreens no longer exist.

API: Remove gdk_window_process_all_updates()

It's completely unused since we have a frame clock.

API: gdk: Remove gdk_window_process_updates()

We have a frame clock now.